Summary

This PR fixes hook errors on Windows and ensures consistent cross-platform behavior.

Changes

1. bin/hooks/session-start.js - Cross-platform CLI detection

Problem: The original code only checked one npm global path, which doesn't work on Windows.

Solution:

• Check AGENTFUL_PARALLEL env var first (user override)
• Try multiple npm global paths for Windows, macOS (Homebrew), and Linux
• Default to enabled: true if CLI not found (newer Claude Code versions have parallel execution by default)

const possiblePaths = [
  // Windows npm global
  path.join(execSync('npm root -g', { encoding: 'utf8' }).trim(), '@anthropic-ai', 'claude-code', 'cli.js'),
  // Unix npm global
  '/usr/local/lib/node_modules/@anthropic-ai/claude-code/cli.js',
  // Homebrew on macOS
  '/opt/homebrew/lib/node_modules/@anthropic-ai/claude-code/cli.js',
];

2. bin/hooks/architect-drift-detector.js - Exit code handling

Problem: The hook exited with code 1 when drift was detected, which Claude Code interprets as a hook error.

Solution: Always exit with code 0. Drift detection is informational, not an error.

// Before
process.exit(driftDetected ? 1 : 0);

// After
process.exit(0);  // Always 0 - drift is informational, not an error

3. template/.claude/settings.json - Correct matcher syntax

Problem: PreToolUse hooks used "tools": ["Write", "Edit"] which is incorrect syntax and causes errors for tools not in the list (Glob, Read, etc.).

Solution: Use the documented matcher syntax with regex patterns:

// Before
"PreToolUse": [
  {
    "tools": ["Write", "Edit"],
    "hooks": [...]
  }
]

// After
"PreToolUse": [
  {
    "matcher": "Write|Edit|NotebookEdit",
    "hooks": [...]
  }
]

Testing

All hooks tested on Windows 11:

✅ session-start.js - Exit 0 (shows "parallel execution: ON")
✅ architect-drift-detector.js - Exit 0 (drift is informational)
✅ All other hooks - Exit 0

No more PostToolUse:Glob or PostToolUse:Read errors.

Related Issues

Fixes hook errors reported by Windows users when using Glob/Read tools.